[Improve] Unify built-in and custom automations - #2451
Conversation
|
No new code issues found. See task
Reviewed dfd4138 |
|
Addressed the latest lock-recovery review chain (original finding, follow-up race) in dfd4138. Filesystem leases were removed; shared settings suites now use a transaction-scoped PostgreSQL advisory lock with database-owned atomic contention and automatic release. Four shared settings suites passed 67/67, web type checking and scoped lint passed, and pre-push gates passed. GitHub CI run 34477823324 completed successfully for exact SHA |
| </span> | ||
| </> | ||
| } | ||
| actions={ |
There was a problem hiding this comment.
This action set no longer includes the task-history link: the historyFilter calculation and its link were removed, with no replacement in the new list. Custom-automation owners can now see only a relative “Last run” timestamp and cannot open their prior runs from this page. Keep the filtered task-history action so the refactor preserves that workflow.
There was a problem hiding this comment.
Bruno clarified the intended right-side action set as Run now, Configure, and Delete only for custom automations. The history action is therefore intentionally omitted from this list rather than restored; the thread is left unresolved as a documented scope decision.
What changed
Why this change was made
The Automations page split related configuration across different layouts and gave built-ins less at-a-glance context than custom automations. A single searchable list makes the full accessible catalog easier to scan without changing who can see or manage each record.
Impact
Admins see built-in and custom automations together, while members continue to see only the custom automations they own and no unavailable Built-in filter. Existing authorization, destination resolution, configuration dialogs, and mutation permissions are unchanged.
The branch now includes current
develop; the previously failing Test job passes on the resulting merge head. Browser proof was not rerun for the test-only CI stabilization change.